Skip to content

Conversation

aechan
Copy link

@aechan aechan commented Oct 16, 2025

The Azure OpenAI Batch API exposes a few extra fields to support batch processing uploads from Blob Storage. This PR simply adds those optional fields to support the Azure OpenAI Batch API.

https://learn.microsoft.com/en-us/azure/ai-foundry/openai/how-to/batch-blob-storage?tabs=rest

@Copilot Copilot AI review requested due to automatic review settings October 16, 2025 17:33
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for Azure OpenAI Batch API's blob storage integration by introducing optional fields for blob-based batch processing. The changes enable users to specify blob storage locations for input data and output destinations when working with Azure's batch processing capabilities.

  • Added blob storage input and output folder properties to batch request schema
  • Added blob storage URL properties to batch response schema for input, output, and error locations
  • Created new BatchOutputFolder class to encapsulate output folder configuration

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
OpenAI-DotNet/Batch/CreateBatchRequest.cs Added InputBlob and OutputFolder properties for blob storage integration
OpenAI-DotNet/Batch/BatchResponse.cs Added ErrorBlob, OutputBlob, and InputBlob properties to track blob storage URLs
OpenAI-DotNet/Batch/BatchOutputFolder.cs New class defining the structure for batch output folder configuration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@StephenHodgson
Copy link
Member

Thanks @aechan I'll take a look at this and try to integrate it in with the next release

@StephenHodgson StephenHodgson changed the base branch from main to development October 19, 2025 14:49
Comment on lines +49 to +52
public string InputBlob { get; set; }

[JsonPropertyName("output_folder")]
public BatchOutputFolder OutputFolder { get; set; }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be set in constructor, but I can handle that tweak

@StephenHodgson StephenHodgson merged commit 4927407 into RageAgainstThePixel:development Oct 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants